Skip to content

Conversation

Nik-Kras
Copy link

@Nik-Kras Nik-Kras commented Nov 8, 2024

No description provided.

@ExtReMLapin
Copy link
Contributor

Average 'I can't code but I want to be in the contributors list' PR

@richdougherty
Copy link
Contributor

richdougherty commented Nov 24, 2024

Hi @Nik-Kras, I think it's nice to keep the text clean. My suggestion is that I would probably go lighter on the changes and just stick to obvious grammar errors (The model will will format), and not just minor style changes. This is because the changes add extra diffs to the code which can make history harder to trace. If you stick to fixes it will also make it easier for the maintainer to merge.

I'm reminded of this first PR on a project I worked on, which got a bit famous with a blog post: playframework/playframework#616. Hopefully that will encourage you.

@devashishraj
Copy link

readme > llm > PR

@jroper
Copy link

jroper commented Sep 19, 2025

@ExtReMLapin This kind of hostility to new contributors to a project is never appropriate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants